fixed the bug that LstmNonlinearityComponent::ZeroStats() is not imp…#1302
fixed the bug that LstmNonlinearityComponent::ZeroStats() is not imp…#1302danpovey merged 2 commits intokaldi-asr:masterfrom
Conversation
|
I ran the experiments on ami sdm1. With the bug fixed, the "fast" version of tdnn_lstm+chain is slightly worse than the non-fast one, while the "fast" version of blstm+chain is slightly better than its non-fast counterpart. The fast and non-fast might be still not totally comparable because of the way the stats are zeroed as mentioned by @danpovey in the issue #1312 |
|
There are other differences between the fast and non-fast ones.
Do you have a baseline that's the same thing before you fixed the bug?
In any case I would expect very little or no difference; the prior is high
that it won't make any difference.
…On Mon, Jan 9, 2017 at 1:18 PM, Yiming Wang ***@***.***> wrote:
I ran the experiments on ami sdm1. With the bug fixed, the "fast" version
of tdnn_lstm+chain is slightly worse than the non-fast one, while the
"fast" version of blstm+chain is slightly better than its non-fast
counterpart.
dev eval
fast tdnn_lstm 38.0 41.3
non-fast tdnn_lstm 37.8 40.8
fast blstm 39.5 42.9
non-fast blstm 39.7 43.3
The fast and non-fast might be still not totally comparable because of the
way the stats are zeroed as mentioned by @danpovey
<https://github.com/danpovey> in the issue #1312
<#1312>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1302 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADJVu_wxy0PTZ8sL6i6FTFaXaYaNrq-Vks5rQqQggaJpZM4LZTuD>
.
|
|
fast version before bug fix: fast tdnn_lstm 37.7 40.6 |
|
Please show the output of chain_dir_info.pl for the systems before and
after the fix, and also show me the directory locations.
…On Mon, Jan 9, 2017 at 1:46 PM, Yiming Wang ***@***.***> wrote:
fast version before bug fix:
fast tdnn_lstm 37.7 40.6
fast blstm 39.7 42.9
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1302 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADJVuwBVaXOuWE1-Shf0fYB_MGdfFSV8ks5rQqqegaJpZM4LZTuD>
.
|
|
tdnn_lstm before/after fix: blstm before/after fix: The path is: |
|
I couldn't find any obvious reason why the WER would be worse [and the optimization was better].. I'm pretty sure this is just random noise we're seeing. And it's a clear bug-fix. So I'm merging. |
…lemented (resolved issue #1301 )
Test is running...